Skip to content

doc: Combine all PPP pages into one "Cellular PPP modem" document#165

Merged
SeppoTakalo merged 1 commit into
nrfconnect:mainfrom
SeppoTakalo:ppp_doc
Feb 11, 2026
Merged

doc: Combine all PPP pages into one "Cellular PPP modem" document#165
SeppoTakalo merged 1 commit into
nrfconnect:mainfrom
SeppoTakalo:ppp_doc

Conversation

@SeppoTakalo

@SeppoTakalo SeppoTakalo commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Describe the PPP behavior in one page, instead of spreading the information between two separate pages.

Remove separated pages for Linux and Zephyr.

Jira: SM-59

@MarkusLassila MarkusLassila left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

Some tuning is required, but this is moving to correct direction. We should agree on whether we use the term dial-up or not.

Comment thread doc/app/at_cmux.rst Outdated
Comment thread doc/app/sm_logging.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/images/dialup-ppp-cmux.svg
Comment thread doc/app/sm_cellular_modem.rst
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Zephyr's cellular modem driver support has some limitations when used with an nRF91 Series SiP:

* GNSS functionality is not available through the driver.
* eDRX or PSM configuration through the driver is not supported. Requires manual changes of the modem settings through AT commands.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not currently possible? The changes should be done in dial-script in the cellular modem.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should also likely mention the AT%XSYSTEMMODE to setup the correct operation mode (LTE-M /NB-IoT).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added here a note regarding that:

* System mode is not configured. See the `%XSYSTEMMODE`_ command in the AT command Reference Guide for more details.

I feel like this is not the right document to instruct how to modify the AT script from Zephyr source code.

Comment thread doc/app/sm_logging.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated

@trantanen trantanen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good overall!

Comment thread doc/app/at_cmux.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst Outdated
Comment thread doc/app/sm_dialup_ppp_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst Outdated
@SeppoTakalo SeppoTakalo force-pushed the ppp_doc branch 6 times, most recently from 5da161d to 6c33001 Compare February 10, 2026 12:45
Comment thread doc/app/at_cmux.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst
Comment thread doc/app/sm_cellular_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst Outdated
Comment thread doc/app/sm_cellular_modem.rst
@SeppoTakalo SeppoTakalo force-pushed the ppp_doc branch 2 times, most recently from e4d37f9 to a191216 Compare February 10, 2026 20:03
@trantanen trantanen changed the title doc: Combine all PPP pages into one "Dial-up modem" document doc: Combine all PPP pages into one "Cellular PPP modem" document Feb 11, 2026
@trantanen

Copy link
Copy Markdown
Collaborator

I renamed the PR for "Cellular PPP modem". Maybe rename the commit as well.

Describe the PPP behavior in one page, instead of spreading the
information between two separate pages.

Remove separated pages for Linux and Zephyr.
Move trace-collecting through CMUX to a logging page.

Signed-off-by: Seppo Takalo <seppo.takalo@nordicsemi.no>
@SeppoTakalo SeppoTakalo merged commit 1cb0c72 into nrfconnect:main Feb 11, 2026
2 checks passed
@SeppoTakalo SeppoTakalo deleted the ppp_doc branch February 11, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants